Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Add error catching for renderTemplate of index.html #28

Merged
2 commits merged into from
Jun 8, 2024

Conversation

MatthewJohn
Copy link
Contributor

No description provided.

releases-generator/api.go Outdated Show resolved Hide resolved
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @MatthewJohn I believe one of your commits is missing a signoff. Could you add one please?

@MatthewJohn
Copy link
Contributor Author

Hey @MatthewJohn I believe one of your commits is missing a signoff. Could you add one please?

Sorry, which commit? I didn't have a sign-off for the original (how would I go about that?)
The second does because I used the suggestion-committing in github :)

@cam72cam
Copy link
Member

cam72cam commented Jun 5, 2024

@MatthewJohn
Copy link
Contributor Author

a08f63c is missing the signoff

https://stackoverflow.com/questions/25570947/how-to-use-git-interactive-rebase-for-signing-off-a-series-of-commits

Ah okay - so it will sign it off as myself?

@ghost
Copy link

ghost commented Jun 5, 2024

@MatthewJohn yes, please sign off with the email address and name you use for your commits. Thank you!

@MatthewJohn
Copy link
Contributor Author

@MatthewJohn yes, please sign off with the email address and name you use for your commits. Thank you!

All done :)

@ghost ghost merged commit 0a88e05 into opentofu:main Jun 8, 2024
95 checks passed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants